GktPlacesSidebar: Plug a ton of leaks

If you don't keep in mind that g_object_get() returns references to
objects and copies fo strings, things leak all over the place.

sidebarrow: Avoid object-valued properties

They tend to leak...as these do.

render: Fix ridge/groove border rendering

The typo strikes again!

GtkHeaderBar: reduce minimum title width

The minimum title width affects the minimum window width
for CSD windows. To allow smaller windows like without
CSD reduce it a bit (276px vs 156px min width)

https://bugzilla.gnome.org/show_bug.cgi?id=751341

GtkWindow: fix default empty window size with CSD

In the non-CSD case we checked for 0x0 window size requisition
and replaced it with 200x200 so the window was still visible.
This no longer works in case of CSD as the shadow and title bar
are always added to the requisition resulting in a titlebar/shadow
only window in case there is no child widget (this is currently
visible under wayland or when setting GTK_CSD=1).

Instead of special casing the final window size, special case
the child requisition paths instead. This gives us the same
requisition in both, CSD and non-CSD cases (the header bar
has a too large minimum width atm so the resulting window is
still not the same)

https://bugzilla.gnome.org/show_bug.cgi?id=751341

mir: fix toggling between maximized and restored

mir: fix wheel scrolling in both directions

mir: emit GDK_ENTER_NOTIFY on pointer enter

mir: remove non-useful logging

mir: don't need dummy egl surface

mir: emit configure event on window move/resize

Without this, some windows don't restart the paint clock needed to
actually render the contents.

mir: fix build due to missing api

Adwaita: scale marks visible on the dark variant

Updated Hungarian translation

GtkScale: More work on mark spacing

Try harder not to push the scale out of balance by marks. With
this change, the remaining difference in the drawing is down
to asymmetric assets and margins coming out of the theme.
To fully support such asymmetric rendering, we need implement
baseline alignment for scales.

https://bugzilla.gnome.org/show_bug.cgi?id=749650

printing: Get output bin via IPP

Request "output-bin-supported" and "output-bin-default" attributes through
IPP if there is no PPD for selected printer.
Pass "output-bin" option with other options in printer_get_options().
Translate standard IPP values of "output-bin" option.

https://bugzilla.gnome.org/show_bug.cgi?id=725441

GtkStack: Avoid some unnecessary complication

We already have an in-destruction flag, so use it.

Fix calculation of edge input window size in case the real window is too small.

This error resulted in warnings like
"pixman_region32_init_rect: Invalid rectangle passed"

In case the window is smaller than handle_size * 2 the resulting
edge window got a negative size. Prevent that by limiting the
handle size to half the respective edge length. This also
prevents the corner windows from overlapping in case the window
is too small.

Revert "window: Check if we can use CSD before enabling them"

This reverts commit c5e5ee67490e7e7af56052d8f8beb75db002c2f1.

It turns out that this commit breaks the default sizing of GtkWindow,
which in turn causes the window to be smaller than the handle size,
which ends up sending invalid rectangles to Cairo.

We'll have to find another way to fix the CSD issue.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=751140

https://bugzilla.gnome.org/show_bug.cgi?id=750343

Avoid spurious operations on destroy

When the stack is destroyed we do not want to waste time running
animations and notifying listeners about which is our current
visible child.

This is not only an optimization, but it is important for the stack
switcher widgets: since they are in another branch of the hieratchy
we do not want to get notifications while the stack is being destroyed.

Based on a patch by Paolo Borelli
https://bugzilla.gnome.org/show_bug.cgi?id=724506

Pacify gobject-introspection

gobject-introspection complains about the parameter names of
vfuncs. Why, you ask ? No idea.

GtkRange: Improve zoom handling a bit

The calculation to update the initial slider position on zoom
changes was not working correctly when using keys to toggle
zoom on and off for scales. Avoid it by updating the position
beforehand.

GtkRange: Make drag gesture less agressive

Moving the mouse while pressing one of the steppers was making
the slider jump to the end, unexpectedly. This was caused by the
drag gesture kicking in when it shouldn't. Fix this by making
all drag gesture signal handlers only do something if we are in
a drag thats started on the slider.

https://bugzilla.gnome.org/show_bug.cgi?id=751050

file chooser: Avoid tall combos

The filter combo box in the file chooser was stretching to match
the height of the extra widgets next to it. Since, stretched combo
boxes are not a pretty sight, make it stick to the top instead.

Patch by Caolán McNamara,

https://bugzilla.gnome.org/show_bug.cgi?id=751227

GtkOverlay: Sync up parameter names

gtk-doc gets confused if the parameter names between header,
source and doc comment don't match up.

Fix a typo

gtk-doc complained about a multi-line Since: tag, but the
real culprit was a typo in the Deprecated: tag.

Tell gtk-doc to ignore private headers

Tell gtk-doc to ignore new private headers as well, otherwise
it complains about undocumented symbols in them.

GtkWindow: Small documentation addition

GtkTextView: Small documentation addition

Fix up index for some 3.14 symbols

Marking these as "Since: 3.14.1" may be more accurate, but
it causes gtk-doc to not put these symbols in any versioned
index at all (it generates an unused index for 3.14.1). So,
lets lie a little, and say these symbols were added in 3.14.

gtk-demo: Add forgotten source file

The font_features.c file was not added as a resource, so that
gtk3-demo could not display it.

gtk-demo: Many cosmetic fixes

Mostly update window titles, and some addition to descriptions.

GtkAssistant: Fix criticals on dispose

After the recent change thta introduced boxes between the pages
and the notebook, we were no longer careful enough when disposing
the assistant. Fix that up.

gtk3-demo: Improve the overlay demo

Add some more information to the description.

gtk-demo: A better overlay demo

This shows the new passthrough feature.

Updated German translation

Updated POTFILES.skip

gtk-demo: Don't Distribute demos.h

Since demos.h is now generated according to the platform for which GTK+ is
built, don't distribute it. Generate a Windows-specific demos.h.win32 and
distribute that instead, in which the Visual Studio build files will copy
it to demos.h, so that the build will proceed normally.

https://bugzilla.gnome.org/show_bug.cgi?id=749622

Bump Pango requirement

The font features demo that was just added uses bleeding-edge
pango features.

Add a font features demo

This started life as a standalone tool, but it is nicer
to have it integrated in gtk3-demo.

file chooser: Small cleanup

Ask the search model if it is empty, instead of maintaining this
information manually in the file chooser widget.

GtkSearchEngineTracker: Fix SPARQL syntax

The function is called tracker:uri-is-descendant, not
tracker-uri-is-descendant.

Forgotten file

GtkSearchEngineSimple: Avoid a private struct

We can just make this type final, and avoid the private struct.

GtkSearchEngineTracker: Avoid a private struct

We can just make this type final, and avoid the private struct.

Avoid an unnecessary check

We create the indexed_locations array unconditionally.

GtkSearchEngine: Avoid a crash

Add a destroy notify for the data of the callback, so we don't
end up leaving a dangling pointer behind for a short while if
the native engine is finalized before the simple one. This
was showing up as crash when typing and backspacing in the
search entry of the file chooser.

GtkSearchEngine: Avoid crawling indexed locations

Implement the IsIndexed callback for tracker. This requires
reading settings of the tracker file miner. We are careful
to avoid a hard dependency on the tracker schemas.

GtkSearchEngine: Avoid crawling indexed locations

Add a framework to the simple engine that allows to skip
locations which are indexed by the native engine.

GtkSearchEngine: Initialize recursive flag

Otherwise all our searches become non-recursive, which was not
the intention.

GtkSearchEngine: Add recursive flag

Add a flag for recursive search, and implement non-recursive
search in both the tracker and simple search engines.

This is not currently used in the file chooser.

Remove an unused field

GtkSearchEngine: Remove unused code

Nothing ever uses or emits the hits-subtracted signal.
Nautilus dropped it a few years ago. Time to follow suit.

Use infos from search engine instead of getting them again

When the search engine provides hits with GFileInfo, use that
to add the hits to the model directly, without going through
another round of async get_info calls.

To do this, we add a batched variant of the
_gtk_file_system_model_update_file call that takes lists of
GFiles and GFileInfos. Again, we can avoid repeated resorting
that happens when the files are updated individually.

Fix an oversight in the previous commit

We lost the line marking the search model as non-empty, causing
us to show the 'no results' page despite there being results.

Request the right attributes in the simple search engine

This change makes it so that the file info passed along
from the simple search engine has all the attributes that
the file system model wants.

search engine: Pass file infos along for hits

file chooser: Use batched insertion for search results

This helps avoiding excess work when procesing many search
results, as typically happens with short search strings
and recursive search.

https://bugzilla.gnome.org/show_bug.cgi?id=751169

file system model: Add batched insertion

Add a batched version of gtk_file_system_model_add_and_query_file
that takes a list of files and avoids resorting the model for each
individual insertion. The querying is still done one-file-at-a-time,
so more optimization is certainly possible.

Revert "window: Reinstate logic for should_use_csd()"

This reverts commit f043fd532dc66a7abb1d4ea57c666655330cc429.

The whole point of having can_use_csd() return TRUE when GTK_CSD is
unset is to use it inside enable_csd().

https://bugzilla.gnome.org/show_bug.cgi?id=751140

window: Reinstate logic for should_use_csd()

The old should_use_csd() function would return FALSE if the GTK_CSD
environment variable is unset; the change in commit c5e5ee67490e7e7
made it return TRUE if GTK_CSD is unset. This has a cascade effect
on the window size, which causes invalid rectangles to bubble down
to Pixman.

https://bugzilla.gnome.org/show_bug.cgi?id=751140

gtkdialog: Remove 2px border

messagedialog: remove 2px border

filechooserdialog: Remove space between window and content

Was removed in 232d6f84f0495e9effedc16d700fc13f5a6e302.

Adwaita: indentation

- lapo was shouting at me.

Adwaita: volume button and scale button padding

I was assuming that GtkVolumeButton and GtkScaleButton could only
contain a icon, now they get the image-button padding by default,
but if the text-button class is applied it is not overridden.

Adwaita: scale popup quick fix

Adwaita: markup cleanup

- move touch selection selectors to its own section
- use scss style markup for .button common elsewhere
  (more compact and legible)

HC: touch text selection styling

- sync with Adwaita
- try to avoid using image assets

https://bugzilla.gnome.org/show_bug.cgi?id=750883

Fix return value error in _gtk_window_get_popover_parent

Cosmetic change

GtkPlacesSidebar: Fix rtl appearance

Apply the necessary directional variants for asymmetric
css padding.

https://bugzilla.gnome.org/show_bug.cgi?id=751084

overlay: remove gtk_overlay_add_pass_through_overlay()

The API to access this functionality will be the setter we just added in
the previous commit.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

overlay: add setters and getters for pass-through child property

This will make the API easier to use from bindings too.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

docs: add new GtkOverlay methods to gtk3-sections.txt

https://bugzilla.gnome.org/show_bug.cgi?id=750568

configure: test have_docbook_style != yes, not have_docbook_dtd

file chooser: Drop unused variables

file chooser: Use a revealer

Use a revealer to manage the visibility of the header area where
we show the pathbar, the location entry or search. This is a bit
smoother, and makes search more similar to a search bar.

texthandle: Avoid double rendering of background/frame

gtk_render_handle() already renders background/frame itself, avoid
doing this twice.

gtkmain: Ignore grab for events in child popovers

Popovers may be spawn when there's GTK+ grabs somewhere else (eg.
text selection popover/handles in an entry in a modal popover). When
this happens, events go to the grab widget (in this case the modal
popover) and are effectively ignored by the event widget, even though
it's can be conceptually a child of the grab widget.

To get away with this, tweak a bit gtk_main_do_event(), so events going
to popovers that are related to grab_widget or a child of it are received,
as it would happen with regular children of grab_widget.

https://bugzilla.gnome.org/show_bug.cgi?id=750993

window: Add concept of popover "parent"

This will be the widget that the popover relates to (::pointing-to in
GtkPopover, ::parent in GtkTextHandle).

Additional API to check the popover/parent relationship between widgets
has been added, which will be useful wherever this is necessary in a
generic manner.

https://bugzilla.gnome.org/show_bug.cgi?id=750993

Fix return value error in gdk_window_set_pass_through

https://bugzilla.gnome.org/show_bug.cgi?id=751099

wayland: Always request transient commit when mapping subsurface

A subsurface positioning operation only takes effect when the parent
surfaces state is applied. If a subsurface is mapped and positioned, but
the parent surface state is not immediately committed, the relative
position of the subsurface is undefined and may be placed incorrectly.

To avoid this undefined state, always request that the parent surface
should be committed after mapping a subsurface so that the position
operation will take effect.

https://bugzilla.gnome.org/show_bug.cgi?id=751098

treeview: always pixelcache widget contents

GtkTreeView has a particularly expensive drawing path. This can cause
issues when part of animated widget sequences. Caching the content while
a model is attached helps reduce the number of full redraws during
exposure greatly.

https://bugzilla.gnome.org/show_bug.cgi?id=751082

pixelcache: allow widgets to always require cached content

Some widgets have very expensive drawing paths. So caching the content
can be useful even when not scrolling.

This can help speed up widgets that are part of animation sequences and
thereby go through spurious expose events.

https://bugzilla.gnome.org/show_bug.cgi?id=751082

Updated Hebrew translation

gdkwindow: avoid updating background pattern if it matches previous

Background patterns are often updated when style changes. In many cases,
the new pattern will match the previous. We can optimize out the
invalidation that will occur upon resetting the same pattern.

gtkplacessidebar: improve variable name

gtkplacessidebar: Remove some old comments

We didn't made up our minds until now from some of them, and for
the others no progress has been made or bugs reported, so probably
they are fine.

gtkplacesidebar: use GtkListBox

We were using GTkTreeView in a simple list. Also, as we know,
GtkCellRenderers are not the best way to theme and manipulate
widgets.

So instead use a GtkListBox to modernize the GtkPlacesSidebar,
and in the way clean up some parts of the code (like headings)
which were not used anymore.
Also we don't use a model anymore, since the data is simple
enough to manage it in a subclass of the row itself.

theme: decrease list row transition

It was somewhat annoying given the "trace" of it. Decrease
the time to avoid too much "trace"

gtkfilechooserwidget: show drop hints on gtkplacessidebar

Now with the API addition on gtkplacessidebar, we can show some
hints for the drop targets.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: make insensitive invalid drop targets

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: add new bookmark icon

Following design mockups

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacesidebar: remove obsolete drop state

We didn't use those for some time now.

gtkplacessidebar: add API for show drop hints

It is convenient to allow applications to show all the drop
targets at once. This improves the user experience with drag
an drop.

The new API allows the application to set the gtkplacessidebar
in a mode where invalid drop targets are insensitive and it
adds a "new bookmark" row. This mode is intended to be set
when the application is aware of a dnd operation and needs to
be stopped kwhen the application is aware that dnd operation
was cancelled or ended in a different part than gtkplacesisdebar.

The context parameter is unused in this patch, but will be
used in next patches when the sidebar will use a GtkListBox.
The reason of being unused now is just convenience.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: new bookmark row only at first position

Following the new mockups, put the new bookmark row always
in the first position, so it's easier to drop an item.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

Deal with events from wrong display

GtkInspector is opening a separate display connection, which makes
it more likely that gtk_get_current_event() returns an event from
the "wrong" display.

GtkMenuButton: Make programmatic toggling work again

This was broken in 0796d7b6ff9393746d.

https://bugzilla.gnome.org/show_bug.cgi?id=751018

GtkMenuButton: Avoid direct button access

We were getting ourselves in trouble by casting touch events
to GdkEventButton and poking directly at their internals. Instead,
use GdkEvent API to get what we need.

This fixes a crash when using the gear menu in epiphany with
touch. The same crash also occurred in testmenubutton.